Skip to content

Fix LT-11746: Make Preview in Lexicon Edit aware of publication#776

Merged
jtmaxwell3 merged 2 commits intomainfrom
LT-11746
Mar 24, 2026
Merged

Fix LT-11746: Make Preview in Lexicon Edit aware of publication#776
jtmaxwell3 merged 2 commits intomainfrom
LT-11746

Conversation

@jtmaxwell3
Copy link
Collaborator

@jtmaxwell3 jtmaxwell3 commented Mar 20, 2026

This fixes https://jira.sil.org/browse/LT-11746. I refactored IsObjectVisible and GiveSimpleWarning in XhtmlDocView so that their code could be shared by XhtmlRecordDocView.


This change is Reviewable

@github-actions
Copy link

github-actions bot commented Mar 20, 2026

NUnit Tests

    1 files  ±0      1 suites  ±0   5m 54s ⏱️ +5s
4 074 tests ±0  4 003 ✅ ±0  71 💤 ±0  0 ❌ ±0 
4 083 runs  ±0  4 012 ✅ ±0  71 💤 ±0  0 ❌ ±0 

Results for commit 534746c. ± Comparison against base commit b6ac397.

♻️ This comment has been updated with latest results.

@jasonleenaylor
Copy link
Contributor

Src/xWorks/XhtmlDocView.cs line 230 at r1 (raw file):

		}

		internal static bool IsObjectVisible(int hvoTarget, LcmCache cache, PropertyTable propertyTable, out DictionaryConfigurationController.ExclusionReasonCode xrc)

Since we are exposing this to other callers we should probably guard for null cache and propertyTable.

Copy link
Contributor

@jasonleenaylor jasonleenaylor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jasonleenaylor reviewed 3 files and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on jtmaxwell3).

@jtmaxwell3 jtmaxwell3 merged commit a352074 into main Mar 24, 2026
6 of 7 checks passed
@jtmaxwell3 jtmaxwell3 deleted the LT-11746 branch March 24, 2026 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants